feat(replay): control values + semanticAnnotation on all interactions - #27

packages/tugboat/lib/src/semantics_flags_compat.dart:31

• semanticsCheckedFromFlags references CheckedState.isTrue, but this package supports Flutter >=3.35 and CheckedState is only present in newer SDKs, so this will fail to compile on 3.35. Since checked is treated dynamically already, avoid referencing the enum symbol directly and derive the bool from toString() instead.

  try {
    if (checked.toString().endsWith('.none')) return null;
    if (checked.toString().endsWith('.mixed')) return null;
    return checked == CheckedState.isTrue;
  } catch (_) {

packages/tugboat/lib/src/control_value.dart:52

• When parsing numeric strings, num.tryParse will also accept "NaN"/"Infinity" and produce non-finite doubles, which will later break JSON encoding. Filter those out (or encode them as string tokens) before returning a number kind.

      final asNum = num.tryParse(trimmed);
      if (asNum != null) {
        return TugboatEncodedControlScalar._(kind: 'number', value: asNum);
      }

packages/tugboat/lib/src/control_value.dart:37

• TugboatEncodedControlScalar.encode returns numbers as-is, which can include non-finite doubles (NaN/Infinity). Those values will cause jsonEncode(...) of session payloads to throw, breaking capture/export. Consider encoding non-finite doubles as string tokens instead.

This issue also appears on line 49 of the same file.

    if (raw is num) {
      return TugboatEncodedControlScalar._(kind: 'number', value: raw);
    }

packages/tugboat/lib/src/controller.dart:2307

• recordPointerDown calls resolver.buildTapContext(...) and then immediately calls resolver.controlValueAt(position) / resolver.semanticAnnotationAt(position), which each perform their own hit-test walk. This introduces multiple hit tests per tap-down and can be a noticeable overhead in complex trees. Consider extending buildTapContext (or the underlying hit-test loop) to also return controlValue/semanticAnnotation so the tap path is only walked once.

      target = tapContext.target;
      tapInventory = tapContext.inventory;
      controlValue = resolver.controlValueAt(position);
      semantic = resolver.semanticAnnotationAt(position);